Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: guide issue request through support #2917

Merged
merged 5 commits into from
Sep 25, 2024
Merged

chore: guide issue request through support #2917

merged 5 commits into from
Sep 25, 2024

Conversation

emmenko
Copy link
Member

@emmenko emmenko commented Sep 18, 2024

Same as commercetools/merchant-center-application-kit@d976e65

This is a request from the Support team to guide requests through the support issues pipeline.

PS: the GitHub Discussions will also be disabled.

@vercel Vercel
Copy link

vercel bot commented Sep 18, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
ui-kit ✅ Ready (Inspect) Visit Preview 💬 Add feedback Sep 25, 2024 4:49pm

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 18, 2024 •
edited
Loading

⚠️ No Changeset found

Latest commit: ddf6071

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@ddouglasz ddouglasz requested review from a team, jaikamat, ddouglasz, tylermorrisford, ByronDWall and misama-ct and removed request for a team September 24, 2024 12:49
ddouglasz
ddouglasz approved these changes Sep 24, 2024
View reviewed changes
Copy link
Contributor

@ddouglasz ddouglasz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a small observation I pointed out.
Thanks!

.github/ISSUE_TEMPLATE/config.yml Outdated
# https://help.github.com/en/github/building-a-strong-community/configuring-issue-templates-for-your-repository#configuring-the-template-chooser
blank_issues_enabled: true
contact_links:
- name: commercetools Support
Copy link
Contributor

@ddouglasz ddouglasz Sep 24, 2024 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pointing this out but not sure if this was deliberate. I think needs same update in the app-kit new issue subject too.
Thanks!

Suggested change
- name: commercetools Support
- name: Commercetools Support

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The word commercetools is always lowercase

@ddouglasz ddouglasz merged commit db45e62 into main Sep 25, 2024
7 checks passed
@ddouglasz ddouglasz deleted the nm-issues-through-support branch September 25, 2024 16:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tylermorrisford tylermorrisford tylermorrisford approved these changes

@ddouglasz ddouglasz ddouglasz approved these changes

@jaikamat jaikamat Awaiting requested review from jaikamat jaikamat is a code owner automatically assigned from commercetools/first-contact-team-fe

@ByronDWall ByronDWall Awaiting requested review from ByronDWall ByronDWall is a code owner automatically assigned from commercetools/first-contact-team-fe

@misama-ct misama-ct Awaiting requested review from misama-ct misama-ct is a code owner automatically assigned from commercetools/first-contact-team-fe

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@emmenko @ddouglasz @tylermorrisford